Skip to content

Allow null or empty authorities for DefaultOAuth2User #9380

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

mayur9991
Copy link
Contributor

Make DefaultOAuth2User more inline with other part of
spring-security.
For example,

  • DefaultOAuth2AuthenticatedPrincipal
  • AbstractAuthenticationToken

Closes gh-9366

Make DefaultOAuth2User more inline with other part of
spring-security.
For example,
- DefaultOAuth2AuthenticatedPrincipal
- AbstractAuthenticationToken

Closes spring-projectsgh-9366
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 27, 2021
@jzheaux jzheaux added in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 27, 2021
@jgrandja jgrandja added status: duplicate A duplicate of another issue type: bug A general bug and removed type: enhancement A general enhancement labels Feb 2, 2021
@jgrandja jgrandja added this to the 5.5.0-M2 milestone Feb 2, 2021
@jgrandja
Copy link
Contributor

jgrandja commented Feb 2, 2021

Thank you for the PR @mayur9991. This is now in master as well as backported.

@jgrandja jgrandja closed this Feb 2, 2021
@mayur9991 mayur9991 deleted the gh-9366 branch February 2, 2021 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) status: duplicate A duplicate of another issue type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove notEmpty check for authorities in DefaultOAuth2User
4 participants